Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade arrow-rs to 51.0.0 and extract common dependencies to top level #127

Merged
merged 8 commits into from
Mar 22, 2024

Conversation

robert3005
Copy link
Member

@robert3005 robert3005 commented Mar 20, 2024

No description provided.

gatesn
gatesn previously requested changes Mar 21, 2024
vortex-array/src/array/primitive/compute/as_arrow.rs Outdated Show resolved Hide resolved
vortex-array/src/array/varbinview/compute.rs Show resolved Hide resolved
bench-vortex/Cargo.toml Outdated Show resolved Hide resolved
@robert3005 robert3005 enabled auto-merge (squash) March 22, 2024 13:08
@robert3005
Copy link
Member Author

Got rid of adding u128 type. For now we just take the scalarbuffer as bytes

@robert3005 robert3005 dismissed gatesn’s stale review March 22, 2024 15:01

Removed controversial bits

@robert3005 robert3005 disabled auto-merge March 22, 2024 15:57
@lwwmanning lwwmanning merged commit 0e46433 into develop Mar 22, 2024
2 checks passed
@lwwmanning lwwmanning deleted the rk/updatedeps branch March 22, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants